From: Marco Trevisan (Treviño) <mail@3v1n0.net>
Date: Tue, 24 Nov 2020 18:19:22 +0000 (+0100)
Subject: gdm: Always show fingerprint error messages
X-Git-Tag: archive/raspbian/50.5-1+rpi1~1^2~616^2^2~338
X-Git-Url: https://dgit.raspbian.org/%22http://www.example.com/cgi/%22/%22http:/www.example.com/cgi/%22?a=commitdiff_plain;h=53db4b99b83dd8c885b966742370cc0348cfb6f3;p=gnome-shell.git

gdm: Always show fingerprint error messages

When the login/lock screen is shown the error messages for background
services are always ignored.

However, in case the service is the fingerprint authentication method
we still want to be able to show error messages to inform the user
about what failed, and eventually that the max retries (that may be
different from the login screen configuration) has been reached.

This handles partially the design issue [1] related to the login/lock
screen fingerprint authentication.

Eventually we want to use pam extensions to use clearer and parse-able
messages, however in the case of the fingerprint service we can be sure
that the fprint PAM module will only send errors on auth failures.

[1] https://gitlab.gnome.org/Teams/Design/os-mockups/-/issues/56

Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1652>
---

diff --git a/js/gdm/util.js b/js/gdm/util.js
index c53fbd217..969e12668 100644
--- a/js/gdm/util.js
+++ b/js/gdm/util.js
@@ -549,7 +549,9 @@ var ShellUserVerifier = class {
     }
 
     _onProblem(client, serviceName, problem) {
-        if (!this.serviceIsForeground(serviceName))
+        const isFingerprint = this.serviceIsFingerprint(serviceName);
+
+        if (!this.serviceIsForeground(serviceName) && !isFingerprint)
             return;
 
         this._queueMessage(problem, MessageType.ERROR);